-
Notifications
You must be signed in to change notification settings - Fork 109
Add cometapi integration #363
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
garybadwal
commented
Oct 19, 2025
- Adds integration for Comet API for Chat Generator.
- This integration enable users to leverage chat models to enhance RAG (Retrieval-Augmented Generation) pipelines using Comet API.
bilgeyucel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @garybadwal. Can you double check the links? It seems like there's no such integration
integrations/comet-api.md
Outdated
| ``` | ||
|
|
||
| ## Usage | ||
| You can use [CometAPIChatGenerator](https://docs.haystack.deepset.ai/docs/cometapichatgenerator) as standalone, within a [pipeline](https://docs.haystack.deepset.ai/docs/pipelines) or with the [Agent component](https://docs.haystack.deepset.ai/docs/agent). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| You can use [CometAPIChatGenerator](https://docs.haystack.deepset.ai/docs/cometapichatgenerator) as standalone, within a [pipeline](https://docs.haystack.deepset.ai/docs/pipelines) or with the [Agent component](https://docs.haystack.deepset.ai/docs/agent). | |
| You can use `CometAPIChatGenerator` as standalone, within a [pipeline](https://docs.haystack.deepset.ai/docs/pipelines) or with the [Agent component](https://docs.haystack.deepset.ai/docs/agent). |
|
Hi @bilgeyucel, Thank you for the review. Regarding the links, I’ve submitted an integration PR in Haystack Core Integrations — Comet API Integration PR. The PR is currently under review, and I’ll update the links once it’s merged. Would that be okay? |
|
@garybadwal it's merged now, feel free update this PR! Thanks! |
Thank you so much mpangrazzi, I'll update this PR. |
|
Hi @bilgeyucel, Can you re-review my PR. Now all links are working and updated. |
|
Hi @bilgeyucel, hope you are doing great. Did you got time to review the PR. |
|
Thank you for the PR @garybadwal! |